treeview: Make final

Make GtkTreeSelection final as well.

flow box: Make final

We keep GtkFlowBoxChild subclassable, though.

listbox Make final

We keep GtkListBoxRow subclassable, though.

notebook: Make final

label: Make final

menubar: Make final

toolbar: make final

menu: Make final

Forgotten file

menu tool button: Make final

radio tool button: Make final

toggle tool button: Use standard padding

tool button: Use standard padding

iconview: Make final

stack: Make final

file chooser error stack: Don't derive from stack

We can just have a GtkStack, instead.

scrolled window: Make final

inspector: Stop deriving from scrolled window

places sidebar: Don't derive from scrolled window

We can just have a scrolled window, instead.

Forgotten file

Merge branch 'wip/matthiasc/popup5' into 'master'

Reimplement popups

See merge request GNOME/gtk!847

inspector: Show surface and renderer

GDK W32: Fix indentation and placate GCC

GDK W32: Correctly log popups

GDK W32: Only use owner windows for popups

GTK4 doesn't have WS_CHILD windows anymore, so hWndParent argument
to CreateWindowEx() is always interpreted as the owner window,
not the parent window.

A window with an owner:
* is above the owner in Z-order
* is destroyed when the owner is destroyed
* is hidden when the owner is minimized
This is enforced by the OS.

GTK can only allow this for popup windows.

Desktop window must never[0] be an owner.

[0]: https://devblogs.microsoft.com/oldnewthing/20040224-00/?p=40493

GDK W32: No more child windows

We don't have child windows (the window-inside-a-window kind of windows)
anymore. Remove all the code related to that.

Don't assume that gdk_surface[_move]_resize() is asynchronous

On Windows that call resizes the native window immediately,
and the corresponding GDK event is emitted and processed
before the control is returned to gtk_window_move_resize().
Therefore, update freeze and configure_request_count increment
must happen before the call, not after it.

Add missing include

GDK W32: Don't handle WM_ACTIVATE for popup windows

Popups can't be active or inactive, so emitting GDK events
in response to WM_ACTIVATE makes no sense for these kinds
of GDK surfaces.

The jury is still out on whether we should block (return 0)
or ignore (don't return anything) this message.

Blocking WM_NCACTIVATE (which we currently ignore) is definitely
not an option - it completely breaks input somehow.

Avoid compiler warnings

Avoid priv being flagged as unused on win32.

GDK W32: Adjust to new popup surfaces

1) Handle GDK_SURFACE_POPUP in RegisterGdkClass()
   (for now pretend it's the same as GDK_SURFACE_TOPLEVEL)

2) Remove useless code from GDK_SURFACE_TOPLEVEL case in _gdk_win32_display_create_surface()
   (now there's just GDK_SURFACE_TOPLEVEL there, no need for a type check)

3) Have a separate case for GDK_SURFACE_POPUP and ensure that
   it doesn't get WS_CHILDWINDOW (and neither should GDK_SURFACE_TEMP).

surface: Cosmetics

Move things where they belong.

popover: Don't constantly redraw the beak

We don't want to constantly draw the arrow
if we don't have to. Save the render node
to achieve this.

wayland: Disconnect the frame clock

The frame clock can survive the surface now,
so we need to disconnect.

win32: Disconnect the frame clock

The frame clock can now survive the surface,
so we need to disconnect when the surface goes away.

x11: Disconnect from the frame clock

The frame clock can now survive its surface,
so we need to disconnect signal handlers.

broadway: Disconnect from the frame clock

The frame clock can no survive its surface,
so we need to disconnect signal handlers.

surface: Give backends a chance to disconnect

Call the destroy vfunc before clearing
frame clock and gl context, so backends
have a chance to clean up.

fishbowl: Don't pop up popovers too early

Wayland does not like this.

popover: Stop needles allocations

We were queuing an allocation whenever
the popover is flipped over, unnecessarily,
since we don't change the size of the surface.

This was showing up as popovers being invisible
when flipped over, under X.

surface: Fix the fallback move-to-rect implementation

On X11, the final and flipped rects were reporting
global coordinates. This was showing up as misplaced
popover beaks.

win32: Set surface type early enough

We need to set the surface type before
the frame clock.

surface: Make only toplevels control events

There is no need for popups to connect to the frame
clock to pause and unpause events on the display -
the toplevel already does it.

And don't connect to paint either - handle paint
on popups recursively.

gdk: Maintain popup tree in the frontend

We will need this information here, and we can
share the child lists between various backends.

surface: Make surface-type a property

Make surface-type a construct-only property,
so we can set it before the frame clock.

Adwaita: Remove Emoji completion special case

We can use GtkPopover::has-arrow now.

inspector: Avoid a critical warning

popover: Make it possible to not have an arrow

This will make popovers move flexible for other
uses, such as the Emoji completion popup.

popover: Set input shapes on wayland too

No reason not to do it. Wayland has input shapes.

win32: Set the surface type

This is copying some code from the corresponding
create_surface implementations of the other backends.

IME IM: Be able to handle NULL widget correctly

GDK W32: Fix the code to compile

Somewhat change the order of initialization (to be closer
to what Wayland backend does).

Also remove the wrapper field that is no longer needed -
it used to hold a pointer to the main GdkWindow instance,
which wrapped GdkWin32ImplWindow. Since impls are gone,
nothing is wrapping anything anymore.

Fix a substitution error, where wrong pointer was added
to the hash table. Added a comment to ensure that future readers
(including myself) won't be confused by the fact that we're
inserting a pointer instead of the handle itself.

GDK W32: be able to handle NULL cursor

surface: Don't dispose the frame clock prematurely

Since we are now sharing frame clocks with multiple
surfaces, we can no longer dispose them unconditionally
when a surface goes away. Only do it if we are a
toplevel (without parent).

This was showing up as criticals on exit when opening
and closing any popover in widget factory.

tooltip: fix critical warnings

We might trigger a requery while unrooted.
Deal with it.

Mention gtk_widget_get_toplevel in the migration guide.

Drop gtk_widget_get_toplevel

Not used anymore.

Stop using gtk_widget_get_toplevel

All uses of it can be replaced by gtk_widget_get_root.

Cosmetics

Mention gtk_widget_is_toplevel in the migration guide

Drop gtk_widget_is_toplevel

No longer used.

Stop using gtk_widget_is_toplevel

Mention gtk_widget_get_surface in the migration guide

Drop gtk_widget_get_surface

It is no longer used.

Stop using gtk_widget_get_surface

Replace all uses of gtk_widget_get_surface by
gtk_native_get_surface.

Mention modal->autohide rename in migration guide

Add a test for the property renaming

builder tool: Handle modal->autohide rename

Add a facility for renamed properties,
and use it for GtkPopover::modal.

popover: Rename modal to autohide

This is the term we use for the surface,
and it matches the behavior a bit better
than modal.

Update all callers.

win32: Build fixes

testsuite: Fix some a11y output

This was affected by a recent fix to GtkAboutDialog.

testsuite: Fix popover-related failures

window: Make unrealize work again

GtkWidgets unrealize accesses the frame clock
of the surface, so we need to keep the surface
in place until after we've chained up.

Drop gtk_widget_set_surface

Not used anymore. Only GtkNative's have surfaces.

Drop GtkWidget::surface

Unneeded.

Work toward dropping widget->surface

Drop special-casing of GtkNative in
most widget vfuncs. GtkNative implementations
need to override these anyway.

Remove gtk_widget_get/set_has_surface

These serve no purpose anymore - widgets don't
have surfaces, unless they're a GtkNative.

Add a GdkSurface::parent property

popover: Bring back the beak

Bring back the code that draws an arrow.

x11: Fix handling of frame clock freezes

Now that popups share the frame clock of their
parent, we have to be much more careful about
freezing the clock, since that may stop updates
for another surface.

This commit makes two changes that make the
X11 handling of the frame clock more similar
to the Wayland backend:
- Use gdk_surface_freeze_updates instead of
  gdk_surface_freeze_toplevel_updates to avoid
  affecting the frame clock
- Bail out early in before_paint/after_paint
  if the surface is frozen, to avoid affecting
  the frame clock

Together, these two make the X11 popup surface
type work without freezing updates for the toplevel.

gdk: Inherit the frame clock for popups

With separate clocks, the phases are not coordinated,
which messes with GTKs size allocation machinery treating
the entire widget tree as a whole, and causes us to
run into assertion where popups get drawn before they
are allocated.

Add a testcase for size allocation

This triggers a critical warning in gtk_widget_snapshot.

widget: Remove a popover special case

We never run this since GtkPopover has its
own size_allocate, anyway.

text view: Allocate popovers

places sidebar: Allocate popovers

color swatch: Allocate popovers

entry: Allocate popovers

scale button: Allocate popovers

Call gtk_native_check_resize() from size_allocate,
as is required now. This gets volume buttons closer
to working again (dragging the slider still doesn't
work).

text: Allocate popovers

We need to do the same thing the menu button
does, and call gtk_native_check_resize for
all our popovers

window: Don't inherit cursors across surfaces

I can't think of a case where this is the desired
behavior. So, instead of setting an explicit cursor
on all popups, just stop walking the parents at
surface boundaries.

widget: Optimize resize propagation

When a GtkNative widget is marked as resize_needed,
we need a current position for its parent and we need
the parent to be allocated (so we can position our
surface), but we don't need the parent to be marked
as resize_needed, since the parent size is entirely
independent of the popup size.

widget: Improve warning messages

When we print warnings about a widget, using
gtk_widget_get_name() is slightly better than
G_OBJECT_TYPE_NAME(), since it will give us
the widgets unique name when available.

Add gdk_surface_get_parent back

The api is the same, the parent is different.
This now returns the parent of popup surfaces.

surface: Remove gdk_surface_set/merge_child_input_shapes()

These are useless now that we don't have child surfaces

surface: Remove old debug code

popover: Use GdkSurface::autohide

The new api is made for this case.

surface: Add an autohide property

This api is meant to mimic xdg-popover.grab - we
show the surface, and dismiss it when we get events
on other surfaces. For foreign surfaces, the compositor
handles that for us; for our own, we check outselves
before delivering events to GTK.

broadway: Track popups

This is more or less a 1:1 copy of the X11 code.

x11: Implement popup surfaces

Make them use o-r windows, and move
with their parent.

We do a sort-of ok job on stacking order
here - whenever the parent window gets a
ConfigureNotify, we just restack all popups
directly on top of their parent. This is good
enough to keep popups on top of their parent
while we drag it around, and it gets the popup
to disappear when raising another window on
top of the parent.

wayland: Use popup parents

Make the Wayland move-to-rect implementation
use popup parents.